Skip to content

[MNT] add standalone performance benchmark suite with pre-2.0 baseline - #964

Closed
solegalli wants to merge 2 commits into
mainfrom
benchmarks-suite
Closed

[MNT] add standalone performance benchmark suite with pre-2.0 baseline#964
solegalli wants to merge 2 commits into
mainfrom
benchmarks-suite

Conversation

@solegalli

Copy link
Copy Markdown
Collaborator

Measures fit() and transform() wall-clock time for 40 representative transformers across all modules, on synthetic data of configurable size. Supports saving a baseline and comparing later runs against it, failing when a benchmark regresses beyond a threshold. Not collected by pytest and not run in CI; see benchmarks/README.md.

Baseline recorded before the 2.0 refactoring (polars support, numpy compute) as the reference to verify that the refactoring speeds up the transformers rather than slowing them down.

solegalli and others added 2 commits July 30, 2026 14:19
Measures fit() and transform() wall-clock time for 40 representative
transformers across all modules, on synthetic data of configurable size.
Supports saving a baseline and comparing later runs against it, failing
when a benchmark regresses beyond a threshold. Not collected by pytest
and not run in CI; see benchmarks/README.md.

Baseline recorded before the 2.0 refactoring (polars support, numpy
compute) as the reference to verify that the refactoring speeds up the
transformers rather than slowing them down.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Rebase the branch onto main (which now includes the ArbitraryImputer,
MissingIndicator, MeanImputer, CountEncoder, Winsoriser and
MeanNormalisationScaler renames) and update run_benchmarks.py and the
recorded baseline to reference the new names instead of the deprecated
aliases.

Only the benchmark's case labels and imports change; the recorded
timing values in the baseline JSON are untouched, so --compare keeps
matching every case against the pre-2.0 baseline instead of reporting
the renamed ones as "not in baseline".
@solegalli

Copy link
Copy Markdown
Collaborator Author

This should have been closed in #984

@solegalli solegalli closed this Jul 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant